MyNUnit by MikePuzanov · Pull Request #9 · MikePuzanov/HomeworkSem3

MikePuzanov · 2022-03-28T13:40:29Z

No description provided.

yurii-litvinov · 2022-04-01T10:04:15Z

MyNUnit/AttributesMyNunit/After.cs

+using System;
+
+/// <summary>
+/// Атрибут, которым обозначают методы, которые вызываются после каждого тестом


Suggested change

/// Атрибут, которым обозначают методы, которые вызываются после каждого тестом

/// Атрибут, которым обозначают методы, которые вызываются после каждого теста

yurii-litvinov · 2022-04-01T10:05:16Z

MyNUnit/AttributesMyNunit/After.cs

+    public After()
+    {
+
+    }


Вообще, пустой конструктор сгенерируется и так, можно не писать.

yurii-litvinov · 2022-04-01T10:13:29Z

MyNUnit/AttributesMyNunit/Test.cs

+
+    public string Ignore { get; set; }
+
+    public Test(Type expected, string ignore = null)


Компилятор огорчён происходящим

yurii-litvinov · 2022-04-01T10:17:01Z

MyNUnit/MyNUnit.Test/MyNUnit.Test.cs

+
+public class Tests
+{
+    private readonly MyNUnit myNUnit = new MyNUnit();


Suggested change

private readonly MyNUnit myNUnit = new MyNUnit();

private readonly MyNUnit myNUnit = new ();

yurii-litvinov · 2022-04-01T10:17:58Z

MyNUnit/MyNUnit/MyNUnit.cs

+        var dlls = Directory.GetFiles(path, "*.dll");
+        var tasks = new Task<List<(string, string)>>[dlls.Length];
+        var message = new List<(string, string)>();
+        Parallel.ForEach(dlls,dll => DoTestFromDll(dll, message));


Suggested change

Parallel.ForEach(dlls,dll => DoTestFromDll(dll, message));

Parallel.ForEach(dlls, dll => DoTestFromDll(dll, message));

yurii-litvinov · 2022-04-01T10:28:19Z

MyNUnit/MyNUnit/MyNUnit.cs

+            return;
+        }
+        object classInstance = Activator.CreateInstance(classDll);    
+        Parallel.ForEach(before, method => RunMethodsWithAttributes(method, messagesFromCurrentTest, classInstance));


То же с Before — если их много, могут быть гонки в пользовательском коде. Так что либо запрещать, чтобы их было много, либо исполнять последовательно

yurii-litvinov · 2022-04-01T10:28:46Z

MyNUnit/MyNUnit/MyNUnit.cs

+                }
+                else
+                {
+                    message = ($"Тест {test.Name} не пройден: ожидалось исключения типа {attribute.Expected}", $"Время: {watch.ElapsedMilliseconds} ms");


Suggested change

message = ($"Тест {test.Name} не пройден: ожидалось исключения типа {attribute.Expected}", $"Время: {watch.ElapsedMilliseconds} ms");

message = ($"Тест {test.Name} не пройден: ожидалось исключение типа {attribute.Expected}", $"Время: {watch.ElapsedMilliseconds} ms");

yurii-litvinov · 2022-04-01T10:29:12Z

MyNUnit/MyNUnit/MyNUnit.cs

+            }
+            else if (exception.InnerException.GetType() != attribute.Expected)
+            {
+                message = ($"Тест {test.Name} не пройден: ожидалось исключения типа {attribute.Expected}, возникло {exception.InnerException.GetType()}", $"Время: {watch.ElapsedMilliseconds} ms");


Suggested change

message = ($"Тест {test.Name} не пройден: ожидалось исключения типа {attribute.Expected}, возникло {exception.InnerException.GetType()}", $"Время: {watch.ElapsedMilliseconds} ms");

message = ($"Тест {test.Name} не пройден: ожидалось исключение типа {attribute.Expected}, возникло {exception.InnerException.GetType()}", $"Время: {watch.ElapsedMilliseconds} ms");

yurii-litvinov · 2022-04-01T10:30:25Z

MyNUnit/MyNUnit/MyNUnit.cs

+    private int CheckAttributesInMethod(Attribute[] attribute)
+    {
+        if (attribute.Length > 0)
+        {
+            return 1;
+        }
+
+        return 0;
+    }


Suggested change

private int CheckAttributesInMethod(Attribute[] attribute)

{

if (attribute.Length > 0)

{

return 1;

}

return 0;

}

private int CheckAttributesInMethod(Attribute[] attribute)

=> attribute.Length > 0 ? 1 : 0;

yurii-litvinov · 2022-04-01T10:31:21Z

MyNUnit/MyNUnit/MyNUnit.cs

+        return 0;
+    }
+
+    private void RunMethodsWithAttributes(MethodInfo method, List<(string, string)> messagesForUser, object ClassIntstase)


Suggested change

private void RunMethodsWithAttributes(MethodInfo method, List<(string, string)> messagesForUser, object ClassIntstase)

private void RunMethodsWithAttributes(MethodInfo method, List<(string, string)> messagesForUser, object classInstance)

yurii-litvinov

Всё ещё несколько предупреждений "Cannot convert null literal to non-nullable reference type." при компиляции. А должно компилироваться без предупреждений.

yurii-litvinov · 2022-04-05T11:11:21Z

MyNUnit/MyNUnit/MyNUnit.cs

+    /// </summary>
+    /// <param name="path">путь до .dll</param>
+    /// <returns>Кортеж из (названия теста, результат о нем)</returns>
+    public (string, string)[] RunTests(string path)


Интересно, почему именно массив и зачем ToArray в конце

yurii-litvinov · 2022-04-05T11:15:05Z

MyNUnit/MyNUnit/MyNUnit.cs

+        return count;
+    }
+
+    private void GetAttributesAndDoBeforeAndAfterClass(MethodInfo method, TestAttributes testAttributes)


Название метода всё ещё неправдиво.

yurii-litvinov · 2022-04-05T11:17:56Z

MyNUnit/MyNUnit/MyNUnit.cs

+/// </summary>
+public class MyNUnit
+{
+    private ConcurrentBag<(string, string)> messages;


ConcurrentBag — это неупорядоченное множество. Сообщения могут произвольно перемешиваться, что, наверное, не то, что хотелось.

yurii-litvinov · 2022-04-05T11:19:19Z

MyNUnit/MyNUnit/MyNUnit.cs

+        {
+            return;
+        }
+        messages.Add(($"Проверка тестов из {classFromDll.Name}", ""));


Выше написано, что messages --- это Кортеж из (названия теста, результат о нем). Тут (и везде, где используется messages) что-то не то написано, либо комментарий неправдив.

yurii-litvinov · 2022-04-05T11:21:48Z

MyNUnit/MyNUnit/MyNUnit.cs

+        }
+    }
+
+    private void RunMethodsWithAttributes(List<MethodInfo> methods, object classIntstase)


Suggested change

private void RunMethodsWithAttributes(List<MethodInfo> methods, object classIntstase)

private void RunMethodsWithAttributes(List<MethodInfo> methods, object classInstance)

yurii-litvinov · 2022-04-05T11:22:41Z

MyNUnit/MyNUnit/MyNUnit.cs

+        }
+    }
+
+    private void RunMethodsWithAttributes(List<MethodInfo> methods, object classIntstase)


Тоже неправдивое название, кстати. Этот метод буквально ничего не делает касательно атрибутов.

yurii-litvinov · 2022-04-05T11:25:41Z

MyNUnit/MyNUnit/MyNUnit.cs

+        var types = new Type[] { typeof(Before), typeof(BeforeClass), typeof(Test), typeof(AfterClass), typeof(After)};
+        var typesCount = new Attribute[5][];
+        for (int i = 0; i < types.Length; i++)
+        {
+            typesCount[i] = Attribute.GetCustomAttributes(method).Where(t => t.GetType() == types[i]).ToArray();
+        }
+        return typesCount;


Самодельный IEnumerable.GroupBy :)

yurii-litvinov · 2022-04-05T11:25:53Z

MyNUnit/MyNUnit/MyNUnit.cs

+    private int GetSumOfAttributesInMethod(Attribute[][] array)
+    {
+        var count = 0;
+        for (int i = 0;i < array.Length;i++)


Suggested change

for (int i = 0;i < array.Length;i++)

for (int i = 0; i < array.Length; i++)

yurii-litvinov · 2022-04-05T11:26:39Z

MyNUnit/MyNUnit/MyNUnit.cs

+    }
+    private class TestAttributes


Suggested change

}

private class TestAttributes

}

private class TestAttributes

lykwer added 6 commits December 6, 2021 00:20

add attributes

85f640f

add files

cff246c

Rework MyNUnit

4b8db7e

update appveyor.yml

628057b

update appveyor.yml

abd1327

change path to directory in tests

1f7823a

yurii-litvinov requested changes Apr 1, 2022

View reviewed changes

fix bugs

c3e0440

yurii-litvinov requested changes Apr 5, 2022

View reviewed changes

	/// Атрибут, которым обозначают методы, которые вызываются после каждого тестом
	/// Атрибут, которым обозначают методы, которые вызываются после каждого теста


		public string Ignore { get; set; }

		public Test(Type expected, string ignore = null)

	private readonly MyNUnit myNUnit = new MyNUnit();
	private readonly MyNUnit myNUnit = new ();

	Parallel.ForEach(dlls,dll => DoTestFromDll(dll, message));
	Parallel.ForEach(dlls, dll => DoTestFromDll(dll, message));

	message = ($"Тест {test.Name} не пройден: ожидалось исключения типа {attribute.Expected}", $"Время: {watch.ElapsedMilliseconds} ms");
	message = ($"Тест {test.Name} не пройден: ожидалось исключение типа {attribute.Expected}", $"Время: {watch.ElapsedMilliseconds} ms");

	private void RunMethodsWithAttributes(MethodInfo method, List<(string, string)> messagesForUser, object ClassIntstase)
	private void RunMethodsWithAttributes(MethodInfo method, List<(string, string)> messagesForUser, object classInstance)

	private void RunMethodsWithAttributes(List<MethodInfo> methods, object classIntstase)
	private void RunMethodsWithAttributes(List<MethodInfo> methods, object classInstance)

	for (int i = 0;i < array.Length;i++)
	for (int i = 0; i < array.Length; i++)

Comments

Conversation

MikePuzanov commented Mar 28, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yurii-litvinov left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants